Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: convert forgotten knowledge boss levers to lib #1783

Merged
merged 8 commits into from
Nov 8, 2023
Merged

Conversation

luan
Copy link
Contributor

@luan luan commented Nov 6, 2023

Also adds a /clearcooldown <boss-name>,<player-name> to more easily reset these cooldowns.

@luan luan marked this pull request as draft November 6, 2023 01:00
@luan luan changed the title fix: convert lady tenebris' lever to lib fix: convert forgotten knowledge boss levers to lib Nov 6, 2023
@luan luan marked this pull request as ready for review November 6, 2023 01:38
@luancolombo
Copy link
Contributor

Bom dia fiz todos os testes e teve erro em 3 bosses, erros que considero critico por poderem crashar o servidor, os resultados postei no discord.

@luan
Copy link
Contributor Author

luan commented Nov 6, 2023

Bom dia fiz todos os testes e teve erro em 3 bosses, erros que considero critico por poderem crashar o servidor, os resultados postei no discord.

Posta os erros aqui? Discord fica ruim de seguir as vezes quando to sem tempo pra ler tudo na hora.

@luan luan mentioned this pull request Nov 6, 2023
8 tasks
@luancolombo
Copy link
Contributor

lady tenebris: agora faz a verificacão de cd na alavanca e se o player ta com cd nao entra, porem: nao faz as outras veriificações como por exemplo como na imagem se um player entra e morre, e outro entrar em seguida, mantem os monster e adiciona mais, se o boss tiver vivo adiciona um outro, ou seja cada player que entrar vai enchendo a sala ate o servidor cair, alem disso se alguem tiver enfrentando o boss e outro player puxar a alavanca, ele entra e adiciona mais boss e mosntros, esse erro eu considero critico pois pode derrubar o server
errortenebris

@luancolombo
Copy link
Contributor

Boss Frozen Horror: antes desse PR a alavanca nao funcionava de jeito nenhum, alem desse PR corrigir a alavanca, eu tambem pude testar a luta, aquecendo o ovo e matando o boss em sua forma mortal(melting frozen horror), foram feito os testes de tentar puxar a alavanca com alguem na sala, e nao foi permitido, foi feito teste de matar o boss e tentar ir de novo, nao foi permitido, e feito o teste de deixar o boss vivo sair da sala e em seguida outro player entrar, a sala e resetada. O boss está 100%

@luancolombo
Copy link
Contributor

luancolombo commented Nov 6, 2023

Boss Thorn knight: Feito teste da luta, ele entrou em suas 3 fases perfeitamente antes de morrer, feito testes de tentar entrar na sala com alguem ja enfrentando o boss, nao foi permitido, feito teste de puxar a alavanca estando em cooldown, nao foi permitido, feito teste de deixar boss vivo com seus mobs e entrtar com outro player, a sala e resetada. Duas observações: Quando voce entra com outro player a sala reseta porem o corpo do boss fica na sala, nao sei se isso vai influenciar de algo....e outra coisa que percebi...QUANDO VOCE MATA O BOSS O LOG FICA LIMPO MAS AO MATAR O CAVALO DELE THORN STEEL DA O ERRO NO LOG. Tirando isso, mecanica e alavanca 100%

errorthorn

@luancolombo
Copy link
Contributor

luancolombo commented Nov 6, 2023

Boss Time guardian: esse foram encontrados problemas criticos, a alavanca funciona normalmente, nao permite jogadores entrarem com alguem ja lutando, nao permite entrar com cooldown e reseta a sala se alguem deixar o boss vivo, POREM: a mecanica nao funciona, quando voce entrar na sala deve estar o boss the time guardian que e a versao mais forte e ele deve ficar alternando entre forma de gelo e energy ate morrer, mas quando voce entra ja aparece as 3 formas e o mais estranho e que AS 3 FORMAS DAO LOOT, sendo que o loot deveria cair so na forma principal, seguem as imagens com log de erro tambem.

timeguardianerror1
timeguardianerror2

@luancolombo
Copy link
Contributor

Boss the dragonking: feito todos os testes, mecanica funcionando, as vezes o boss demora um pouco para aparecer com sua forma mortal na outra sala mas isso nao impediu de matar ele, feito testes de alavanca tudo nesse boss ta 100% sem nenhum erro de log.

Boss Lloyd: Ta tudo 100% conforme as correções que ja foram feitas, sem erros de log mecanicas 100%, alavanca 100%

@luancolombo
Copy link
Contributor

Boss The Last LoreKeeper: A alavanca funciona 100%, cd 100%, porem quando voce vai pra sala do boss, o lore keeper ja ta la, e quando voce o mata ele evapora sem dar loot e gera erro no log conforme iimagem, tambem quando voce enrta na sala antes de matar o boss da um erro de monster not exst, eu teste criar esses monstros e todos estao funcionais, testei as positions onde esses monstros nascem e estao corretas a nao ser por essas duas ((((({ monster = "the distorted astral source", pos = Position(31986, 32823, 15) },
{ monster = "an astral glyph", pos = Position(31989, 32823, 15) },)))))

que vao dar nesses quadrados conforme imagem

Eu recomendo enquanto a mecanica nao estiver funcional, ao menos poder matar o boss seria interessante.
errorlore1
errorlore2
timeguardianerror3

@luan luan force-pushed the luan/fix-tenebris branch from df0cabf to 9091206 Compare November 7, 2023 23:58
@luan
Copy link
Contributor Author

luan commented Nov 7, 2023

@luancolombo arrumei tudo que vc reportou (acho)

@luancolombo
Copy link
Contributor

@luancolombo arrumei tudo que vc reportou (acho)

hj nao sei se consigo fazer os testes, vou tentar testar tudo amanha maninho.

@dudantas
Copy link
Member

dudantas commented Nov 8, 2023

@luancolombo arrumei tudo que vc reportou (acho)

hj nao sei se consigo fazer os testes, vou tentar testar tudo amanha maninho.

Se achar bug cria um issue ou reporta no discord, assim abrimos outro pr.

@dudantas dudantas merged commit a9c391b into main Nov 8, 2023
20 checks passed
@dudantas dudantas deleted the luan/fix-tenebris branch November 8, 2023 02:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants